Skip to content

test: add central S-Core PR checks workflow#346

Open
AlexanderLanin wants to merge 2 commits intomainfrom
on_pr
Open

test: add central S-Core PR checks workflow#346
AlexanderLanin wants to merge 2 commits intomainfrom
on_pr

Conversation

@AlexanderLanin
Copy link
Copy Markdown
Member

@AlexanderLanin AlexanderLanin commented Jan 15, 2026

This is just preparation. It does not yet replace any of the other workflows.

We can reduce the number of workflows once this central on_pr workflows handles some of them.

Copilot AI review requested due to automatic review settings January 15, 2026 16:30
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 15, 2026 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 17dabc05-497f-4858-b8de-978329d549ce
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (70 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (73 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (123 packages loaded, 2293 targets configured)

Analyzing: target //src:license-check (132 packages loaded, 2458 targets configured)

Analyzing: target //src:license-check (134 packages loaded, 2484 targets configured)

INFO: Analyzed target //src:license-check (137 packages loaded, 4500 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 14.634s, Critical Path: 0.31s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@AlexanderLanin
Copy link
Copy Markdown
Member Author

AlexanderLanin commented Jan 15, 2026

oh great... guess I need to fix that central workflow first 🤣

Copilot AI reviewed Jan 15, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new centralized GitHub Actions workflow for S-Core PR checks as a preparatory step toward consolidating multiple existing workflows. The workflow calls a reusable workflow from the eclipse-score/cicd-workflows repository.

Changes:

  • Add new score_on_pr.yml workflow that serves as a central entry point for PR checks
  • Configure the workflow to trigger on pull requests, merge groups, and manual dispatch
  • Set up appropriate permissions for automated PR comment creation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/score_on_pr.yml
Comment on lines +36 to +37
# Note: "Common" is what will appear on the website as the job name.
Common:
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The job name 'PR-Checks' uses inconsistent naming convention. Other jobs in the repository use kebab-case (e.g., 'copyright-check', 'formatting-check', 'docs-verify'). Consider renaming to 'pr-checks' to maintain consistency with the existing naming patterns.

Suggested change
# Note: "Common" is what will appear on the website as the job name.
Common:
# Note: the job identifier uses kebab-case to match repository convention.
pr-checks:

Copilot uses AI. Check for mistakes.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 15, 2026

The created documentation from the pull request is available at: docu-html

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 31, 2026

Thanks for the pull request. This thread has been quiet for 30 days, so we are marking it as stale for now. Please take a quick look and let us know whether it is still up to date, still relevant, needs review, or is ready to merge. Any new activity will remove the stale label automatically. If nothing changes in the next 10 days, we will close it to keep the backlog current.

@github-actions github-actions bot added the stale label Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@MaximilianSoerenPollak MaximilianSoerenPollak Awaiting requested review from MaximilianSoerenPollak MaximilianSoerenPollak is a code owner

@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix is a code owner

@nradakovic nradakovic Awaiting requested review from nradakovic nradakovic is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

stale

Projects

INF - Infrastructure Community
Status: Backlog

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AlexanderLanin